Skip to content

Use clean internal link when server mod_rewrite_working is true#17251

Open
MaqsudMallick wants to merge 4 commits into
nextcloud:masterfrom
MaqsudMallick:ingestModRewriteWorking
Open

Use clean internal link when server mod_rewrite_working is true#17251
MaqsudMallick wants to merge 4 commits into
nextcloud:masterfrom
MaqsudMallick:ingestModRewriteWorking

Conversation

@MaqsudMallick

@MaqsudMallick MaqsudMallick commented Jun 24, 2026

Copy link
Copy Markdown

🖼️ Screenshots

🏚️ Before 🏡 After
image image

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not needed
  • 🔙 Backport requests are created or not needed: /backport to stable-xx.x
  • 📅 Milestone is set
  • 🌸 PR title is meaningful (if it should be in the changelog: is it meaningful to users?)

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

Fixes #13071
Depends on nextcloud/android-library#2057

Signed-off-by: MaqsudMallick <maqsudmallick@gmail.com>
Signed-off-by: MaqsudMallick <maqsudmallick@gmail.com>
* SPDX-License-Identifier: AGPL-3.0-or-later
*/

package com.nextcloud.client.database.migrations

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be working with auto migration.
Can you please test this?
This would omit the manual migration.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have tested with the auto migration. It is working.
Removed the manual migration in favor of auto migration.

Signed-off-by: MaqsudMallick <maqsudmallick@gmail.com>
@MaqsudMallick MaqsudMallick force-pushed the ingestModRewriteWorking branch from 073c93c to ba09e03 Compare July 2, 2026 18:14
…king

Signed-off-by: MaqsudMallick <maqsudmallick@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Share links don't use pretty URL format

3 participants